Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docs: support for multi-language tabs #231

Merged
merged 2 commits into from
Jul 4, 2018
Merged

Conversation

eladb
Copy link
Contributor

@eladb eladb commented Jul 4, 2018

Installs the sphinx-tabs extension which allows creating
tabbed code snippets:

.. tabs::

   .. code-tab:: c

         int main(const int argc, const char **argv) {
           return 0;
         }

   .. code-tab:: c++

         int main(const int argc, const char **argv) {
           return 0;
         }

Added Java to the welcome page:

screen shot 2018-07-04 at 1 41 26 am

Installs the [sphinx-tabs] extension which allows creating
tabbed code snippets:

```
.. tabs::

   .. code-tab:: c

         int main(const int argc, const char **argv) {
           return 0;
         }

   .. code-tab:: c++

         int main(const int argc, const char **argv) {
           return 0;
         }
```

Added Java to the welcome page.

[sphinx-tabs]: https://github.com/djungelorm/sphinx-tabs
@rix0rrr
Copy link
Contributor

rix0rrr commented Jul 4, 2018

That is pretty sweet!

import { App, Stack, StackProps } from '@aws-cdk/core';
import { Topic } from '@aws-cdk/sns';
import { Queue } from '@aws-cdk/sqs';
.. code-tab:: js
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The tab says JavaScript but it's TypeScript. Problem?

Copy link
Contributor

@RomainMuller RomainMuller left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Love this! But I share @rix0rrr's comment on how the js tab should be a ts tab (and hopefully the syntax highlighter knows what to do?)

@eladb
Copy link
Contributor Author

eladb commented Jul 4, 2018

Fixed:

screen shot 2018-07-04 at 2 18 25 am

@eladb eladb merged commit c3503f8 into master Jul 4, 2018
@eladb eladb deleted the benisrae/docs-multi-language branch July 4, 2018 09:22
@NGL321 NGL321 added the contribution/core This is a PR that came from AWS. label Sep 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contribution/core This is a PR that came from AWS.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants